Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Changement de conditions de synchronisation des DDB vers le crm brevo #1622

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

madjid-asa
Copy link
Contributor

Quoi ?

Changement de conditions de synchronisation des DDB vers le crm brevo. Au lieu de synchroniser par rapport au montant, on synchronise juste si l'attribut is_followed_by_us=True

Pourquoi ?

Certains cas sont compliqué à jugé sur le prix dans un premier temps.

@madjid-asa madjid-asa requested review from Guilouf and chloend January 2, 2025 12:02
@madjid-asa madjid-asa self-assigned this Jan 2, 2025
lemarche/tenders/tests/test_admin.py Outdated Show resolved Hide resolved
lemarche/tenders/tests/test_admin.py Show resolved Hide resolved
lemarche/tenders/tests/test_admin.py Outdated Show resolved Hide resolved
lemarche/tenders/admin.py Show resolved Hide resolved
Comment on lines +263 to +274
proj_resulted_in_reserved_tender = forms.BooleanField(
widget=forms.CheckboxInput(),
required=False,
label=Tender._meta.get_field("proj_resulted_in_reserved_tender").verbose_name,
)

is_reserved_tender = forms.BooleanField(
widget=forms.CheckboxInput(),
required=False,
label=Tender._meta.get_field("is_reserved_tender").verbose_name,
)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ça sort un peu du cadre de la PR mais c'est un petit quick win pour faciliter la saisie pour les bizdev

lemarche/tenders/models.py Outdated Show resolved Hide resolved
lemarche/tenders/tests/test_admin.py Outdated Show resolved Hide resolved
lemarche/tenders/tests/test_admin.py Outdated Show resolved Hide resolved
@madjid-asa madjid-asa force-pushed the madjid-asa/improve-sync-brevo branch from 82b1a9d to 0b66557 Compare January 7, 2025 09:15
@madjid-asa madjid-asa force-pushed the madjid-asa/improve-sync-brevo branch from 0b66557 to 3e5dad3 Compare January 7, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants